Skip to content

Bump LDK to 49912057 and bitcoin-payment-instructions to e9d7c07d#840

Merged
tnull merged 1 commit intolightningdevkit:mainfrom
tnull:2026-03-bump-ldk
Mar 25, 2026
Merged

Bump LDK to 49912057 and bitcoin-payment-instructions to e9d7c07d#840
tnull merged 1 commit intolightningdevkit:mainfrom
tnull:2026-03-bump-ldk

Conversation

@tnull
Copy link
Collaborator

@tnull tnull commented Mar 24, 2026

Update rust-lightning dependency to rev 49912057895ddfbd6 and bitcoin-payment-instructions to tnull/bitcoin-payment-instructions rev e9d7c07d7affc7714b02. Adapt to API changes:

  • Remove ChainSource generic parameter from LiquidityManager type alias as it was dropped upstream
  • Remove chain_source from LiquiditySourceBuilder and its call sites since LiquidityManager::new no longer accepts it
  • Add new lsps1_service_config field to LiquidityServiceConfig

Co-Authored-By: HAL 9000

@tnull tnull requested a review from joostjager March 24, 2026 15:53
@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Mar 24, 2026

👋 Thanks for assigning @joostjager as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

joostjager
joostjager previously approved these changes Mar 25, 2026
Update rust-lightning dependency to rev 49912057895ddfbd6 and
bitcoin-payment-instructions to tnull/bitcoin-payment-instructions
rev e9d7c07d7affc7714b02. Adapt to API changes:

- Remove `ChainSource` generic parameter from `LiquidityManager` type
  alias as it was dropped upstream
- Remove `chain_source` from `LiquiditySourceBuilder` and its call
  sites since `LiquidityManager::new` no longer accepts it
- Add new `lsps1_service_config` field to `LiquidityServiceConfig`

Co-Authored-By: HAL 9000
@tnull tnull force-pushed the 2026-03-bump-ldk branch from 659809c to ff231e3 Compare March 25, 2026 09:46
@tnull tnull requested a review from joostjager March 25, 2026 12:56
@tnull tnull merged commit a555133 into lightningdevkit:main Mar 25, 2026
24 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants